Skip to content

groups: Add mailing list for MCP Lifecycle Operator#9178

Merged
k8s-ci-robot merged 3 commits into
kubernetes:mainfrom
jaideepr97:mcp-lifecycle-operator-staging-ml
Mar 11, 2026
Merged

groups: Add mailing list for MCP Lifecycle Operator#9178
k8s-ci-robot merged 3 commits into
kubernetes:mainfrom
jaideepr97:mcp-lifecycle-operator-staging-ml

Conversation

@jaideepr97
Copy link
Copy Markdown
Contributor

@jaideepr97 jaideepr97 commented Mar 4, 2026 •
edited
Loading

Requirement for creation of a GCP AR repository

closes kubernetes-sigs/mcp-lifecycle-operator#2

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Mar 4, 2026
@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Mar 4, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Mar 4, 2026

Hi @jaideepr97. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added area/access Define who has access to what via IAM bindings, role bindings, policy, etc. area/groups Google Groups management, code in groups/ size/S Denotes a PR that changes 10-29 lines, ignoring generated files. sig/apps Categorizes an issue or PR as relevant to SIG Apps. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra. labels Mar 4, 2026
@jaideepr97 jaideepr97 marked this pull request as ready for review March 5, 2026 13:34
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 5, 2026
@jaideepr97
Copy link
Copy Markdown
Contributor Author

jaideepr97 commented Mar 5, 2026

@mikebrow @ArangoGutierrez I have included your email IDs here for the lifecycle operator mailing list since you are codeowners
I hope that is okay, happy to change/take them off if not! Please let me know either way

@jaideepr97 jaideepr97 mentioned this pull request Mar 5, 2026
Closed
@aliok
Copy link
Copy Markdown
Contributor

aliok commented Mar 9, 2026

cc @Priyankasaggu11929

@Priyankasaggu11929
Copy link
Copy Markdown
Member

Priyankasaggu11929 commented Mar 9, 2026

/ok-to-test

@aliok, I'm fine with the changes but ideally it should be SIG Apps lead reviewing the changes since the repo is owned by them.

The only thing, I would suggest to cross check is if you want to add more/less people to the mailing list created in the PR.

@kubernetes/sig-apps-leads for review.

Thanks!

@k8s-ci-robot k8s-ci-robot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Mar 9, 2026
Priyankasaggu11929
Priyankasaggu11929 reviewed Mar 9, 2026
View reviewed changes
Comment thread groups/sig-apps/groups.yaml Outdated
- soltysh@gmail.com

- email-id: k8s-infra-staging-mcp-lifecycle-operator@kubernetes.io
name: k8s-infra-staging-mcp-lifecycle-operator
Copy link
Copy Markdown
Member

@Priyankasaggu11929 Priyankasaggu11929 Mar 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://prow.k8s.io/view/gs/kubernetes-ci-logs/pr-logs/pull/k8s.io/9178/pull-k8sio-groups-test/2030962633871462400#1:build-log.txt%3A85

 --- FAIL: TestStagingEmailLength (0.00s)
    groups_test.go:139: Number of characters in project name "mcp-lifecycle-operator" should not exceed 18; is: 22

Copy link
Copy Markdown
Contributor Author

@jaideepr97 jaideepr97 Mar 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, shortened the name

@jaideepr97 jaideepr97 mentioned this pull request Mar 9, 2026
Merged
@k8s-ci-robot k8s-ci-robot added area/artifacts Issues or PRs related to the hosting of release artifacts for subprojects area/infra Infrastructure management, infrastructure design, code in infra/ area/prow Setting up or working with prow in general, prow.k8s.io, prow build clusters area/registry.k8s.io Code in registry.k8s.io/ size/L Denotes a PR that changes 100-499 lines, ignoring generated files. sig/network Categorizes an issue or PR as relevant to SIG Network. sig/node Categorizes an issue or PR as relevant to SIG Node. sig/testing Categorizes an issue or PR as relevant to SIG Testing. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 9, 2026
@jaideepr97
groups: Add mailing list for MCP Lifecycle Operator
73efdad 
Signed-off-by: Jaideep Rao <jrao@redhat.com>
@jaideepr97 jaideepr97 force-pushed the mcp-lifecycle-operator-staging-ml branch from 1e5252d to 2839f56 Compare March 9, 2026 14:45
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Mar 9, 2026
@k8s-infra-ci-robot
Copy link
Copy Markdown
Contributor

k8s-infra-ci-robot commented Mar 9, 2026

Argo CD Diff Preview

Summary:

No changes found

No changes found

Stats:
[Applications: 76], [Full Run: 1m42s], [Rendering: 1m34s], [Cluster: 0s], [Argo CD: 5s]

@jaideepr97
shorten project name in mailing list address
52b3996 
Signed-off-by: Jaideep Rao <jrao@redhat.com>
@jaideepr97 jaideepr97 force-pushed the mcp-lifecycle-operator-staging-ml branch from 2839f56 to 52b3996 Compare March 9, 2026 14:54
Priyankasaggu11929
Priyankasaggu11929 reviewed Mar 9, 2026
View reviewed changes
Comment thread groups/sig-apps/groups.yaml Outdated
@jaideepr97 @Priyankasaggu11929
Update name to match email id
7a3a967 
Co-authored-by: Priyanka Saggu <priyankasaggu11929@gmail.com>
@Priyankasaggu11929
Copy link
Copy Markdown
Member

Priyankasaggu11929 commented Mar 9, 2026

/lgtm
/approve

/hold
(@aliok, feel free to unhold if ready to merge)

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 9, 2026
@k8s-ci-robot k8s-ci-robot added lgtm "Looks good to me", indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 9, 2026
@aliok
Copy link
Copy Markdown
Contributor

aliok commented Mar 10, 2026

I leave the unhold to @mikebrow @ArangoGutierrez

ArangoGutierrez
ArangoGutierrez approved these changes Mar 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@ArangoGutierrez ArangoGutierrez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Mar 11, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ArangoGutierrez, jaideepr97, Priyankasaggu11929

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jaideepr97
Copy link
Copy Markdown
Contributor Author

jaideepr97 commented Mar 11, 2026

/unhold
/remove-hold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 11, 2026
@jaideepr97
Copy link
Copy Markdown
Contributor Author

jaideepr97 commented Mar 11, 2026

@kow3ns @MadhavJivrajani PTAL when you can, would like to get this in as it is blocking #9177

@k8s-ci-robot k8s-ci-robot merged commit 871653a into kubernetes:main Mar 11, 2026
4 checks passed
@k8s-ci-robot k8s-ci-robot added this to the v1.36 milestone Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Priyankasaggu11929 Priyankasaggu11929 Priyankasaggu11929 left review comments

@kow3ns kow3ns Awaiting requested review from kow3ns

@MadhavJivrajani MadhavJivrajani Awaiting requested review from MadhavJivrajani

+1 more reviewer

@ArangoGutierrez ArangoGutierrez ArangoGutierrez approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@ArangoGutierrez ArangoGutierrez

@Priyankasaggu11929 Priyankasaggu11929

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/access Define who has access to what via IAM bindings, role bindings, policy, etc. area/artifacts Issues or PRs related to the hosting of release artifacts for subprojects area/groups Google Groups management, code in groups/ area/infra Infrastructure management, infrastructure design, code in infra/ area/prow Setting up or working with prow in general, prow.k8s.io, prow build clusters area/registry.k8s.io Code in registry.k8s.io/ cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. sig/apps Categorizes an issue or PR as relevant to SIG Apps. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra. sig/network Categorizes an issue or PR as relevant to SIG Network. sig/node Categorizes an issue or PR as relevant to SIG Node. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

v1.36

Development

Successfully merging this pull request may close these issues.

Image repository setup

6 participants

@jaideepr97 @k8s-ci-robot @aliok @Priyankasaggu11929 @k8s-infra-ci-robot @ArangoGutierrez